Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhancement/user field abstraction #3223

Open
wants to merge 23 commits into
base: v3.4
Choose a base branch
from

Conversation

dparker1005
Copy link
Member

All Submissions:

Changes proposed in this Pull Request:

  • Adds a new class PMPro_Field_Group for managing registered PMPro_Field objects.
  • Improves how user field values are displayed
  • Adds deprecation warnings to the PMPro_Field class to enable cleanup in the next major release

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you successfully run tests with your changes locally?

Changelog entry

Enter a summary of all changes on this Pull Request. This will appear in the changelog if accepted.

dparker1005 and others added 19 commits October 29, 2024 15:45
Overhauling user fields logic with abstraction, simplifications, and deprecations
…; adjusting JS to toggle all file fields properly
…y-values

Improving display values for links and files
displayValue case for number or date field types
@dparker1005 dparker1005 requested a review from ideadude December 5, 2024 20:05
Adjusting CSS to better allow for added field settings by custom code.
@dparker1005 dparker1005 added this to the 3.4 milestone Dec 17, 2024
dparker1005 and others added 3 commits January 10, 2025 10:21
…o the element param of pmpro_get_element_class so form processing can target
…t-messages

Logic to add class names based on field ID and passing the field ID to the element param of pmpro_get_element_class so form processing can target
@dparker1005 dparker1005 changed the base branch from dev to v3.4 January 20, 2025 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants